Raise tag limit for objects #30

xla · 2016-12-06T14:12:16Z

With the increased use of tags esp. for semantic distinction of posts the current limit turns out to be not sufficient when semantic tags are mixed with user input. There might be an opportunity to split those apart to avoid unnecessary side-effects.

nilsen340 · 2016-12-06T14:14:13Z

service/object/object.go

@@ -175,7 +175,7 @@ func (o *Object) Validate() error {
 		)
 	}

-	if len(o.Tags) > 5 {
+	if len(o.Tags) > 25 {


Vision in one line

With the increased use of tags esp. for semantic distinction of posts the current limit turns out to be not sufficient when semantic tags are mixed with user input. There might be an opportunity to split those apart to avoid unnecessary side-effects.

xla self-assigned this Dec 6, 2016

nilsen340 reviewed Dec 6, 2016

View reviewed changes

Raise tag limit for objects

9d12ff9

With the increased use of tags esp. for semantic distinction of posts the current limit turns out to be not sufficient when semantic tags are mixed with user input. There might be an opportunity to split those apart to avoid unnecessary side-effects.

xla force-pushed the tag-limit branch from f98df61 to 9d12ff9 Compare December 6, 2016 14:15

nilsen340 approved these changes Dec 6, 2016

View reviewed changes

xla merged commit a85b1f1 into master Dec 6, 2016

xla deleted the tag-limit branch December 6, 2016 14:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Raise tag limit for objects #30

Raise tag limit for objects #30

xla commented Dec 6, 2016

nilsen340 Dec 6, 2016

Raise tag limit for objects #30

Raise tag limit for objects #30

Conversation

xla commented Dec 6, 2016

nilsen340 Dec 6, 2016

Choose a reason for hiding this comment